Skip to content

fix(sa): increase detectorsSearch wait timeout - #2099

Open
yashgupte21 wants to merge 1 commit into
opensearch-project:3.1from
yashgupte21:fix-sa-3.1
Open

fix(sa): increase detectorsSearch wait timeout#2099
yashgupte21 wants to merge 1 commit into
opensearch-project:3.1from
yashgupte21:fix-sa-3.1

Conversation

@yashgupte21

@yashgupte21 yashgupte21 commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Description

Increase detectorsSearch wait timeout from 300s to 500s in the SA detector tests. The 300s timeout is insufficient when running against domains with higher network latency, causing flaky failures in the beforeEach hooks.

Issues Resolved

Resolves flaky cy.wait('@detectorsSearch') timeout in 1_detectors.spec.js

(Run Finished)


       Spec                                              Tests  Passing  Failing  Pending  Skipped  
  ┌────────────────────────────────────────────────────────────────────────────────────────────────┐
  │ ✔  plugins/security-analytics-dashboar      06:03       14       12        -        2        - │
  │    ds-plugin/1_detectors.spec.js                                                               │
  └────────────────────────────────────────────────────────────────────────────────────────────────┘
    ✔  All specs passed!                        06:03       14       12        -        2        -  

✨  Done in 394.84s.

Check List

  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@yashgupte21
fix(sa): increase detectorsSearch wait timeout
ba23d4d 
Signed-off-by: YashPradeep Gupte <yashprg@amazon.com>
@github-actions

github-actions Bot commented Jul 17, 2026

Copy link
Copy Markdown

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🧪 No relevant tests
🔒 No security concerns identified
✅ No TODO sections
🔀 No multiple PR themes
⚡ No major issues detected

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kavilla kavilla Awaiting requested review from kavilla kavilla is a code owner
@ohltyler ohltyler Awaiting requested review from ohltyler ohltyler is a code owner
@CCongWang CCongWang Awaiting requested review from CCongWang CCongWang is a code owner
@ashwin-pc ashwin-pc Awaiting requested review from ashwin-pc ashwin-pc is a code owner
@peterzhuamazon peterzhuamazon Awaiting requested review from peterzhuamazon peterzhuamazon is a code owner
@ananzh ananzh Awaiting requested review from ananzh ananzh is a code owner
@prudhvigodithi prudhvigodithi Awaiting requested review from prudhvigodithi prudhvigodithi is a code owner
@xluo-aws xluo-aws Awaiting requested review from xluo-aws xluo-aws is a code owner
@Hailong-am Hailong-am Awaiting requested review from Hailong-am Hailong-am is a code owner
@SuZhou-Joe SuZhou-Joe Awaiting requested review from SuZhou-Joe SuZhou-Joe is a code owner
@ruanyl ruanyl Awaiting requested review from ruanyl ruanyl is a code owner
@wanglam wanglam Awaiting requested review from wanglam wanglam is a code owner
@zhongnansu zhongnansu Awaiting requested review from zhongnansu zhongnansu is a code owner
@yujin-emma yujin-emma Awaiting requested review from yujin-emma yujin-emma is a code owner
@yubonluo yubonluo Awaiting requested review from yubonluo yubonluo is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@yashgupte21